Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs(readme): switch to dynamic coverage badge #116

Merged
merged 1 commit into from
Jan 29, 2024
Merged

Conversation

cdcabrera
Copy link
Owner

What's included

  • docs(readme): switch to dynamic coverage badge

Notes

  • flips the coverage badge to something more predictable than codcovs badge...
    • currently the codecov badge is borking and we don't really find it enjoyable to figure out what random codecov requirement changed
  • uses shields.io for a dynamic badge built from json
  • badge now only indicates line coverage for the existing release
  • any subsequent coverage changes will be updated when the release script is run, coverage is stored in package.json

How to test

...

Example

...

Updates issue/story

ongoing

@cdcabrera cdcabrera added the documentation Improvements or additions to documentation label Jan 29, 2024
@cdcabrera cdcabrera merged commit aab6323 into main Jan 29, 2024
5 checks passed
@cdcabrera cdcabrera deleted the coverage-badge branch March 21, 2024 13:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant